Skip to content

[libc++][Android] Reenable 2 tests for Android #149415

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 18, 2025

Conversation

rprichard
Copy link
Contributor

Now that the Android clang has been upgraded to clang-r563880 (llvm.org/pr148998), these two tests pass again.

Now that the Android clang has been upgraded to clang-r563880
(llvm.org/pr148998), these two tests pass again.
@rprichard rprichard requested a review from a team as a code owner July 17, 2025 22:03
@llvmbot llvmbot added the libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi. label Jul 17, 2025
@llvmbot
Copy link
Member

llvmbot commented Jul 17, 2025

@llvm/pr-subscribers-libcxx

Author: Ryan Prichard (rprichard)

Changes

Now that the Android clang has been upgraded to clang-r563880 (llvm.org/pr148998), these two tests pass again.


Full diff: https://github.com/llvm/llvm-project/pull/149415.diff

2 Files Affected:

  • (modified) libcxx/test/std/utilities/meta/meta.unary/meta.unary.comp/is_bounded_array.pass.cpp (-3)
  • (modified) libcxx/test/std/utilities/meta/meta.unary/meta.unary.prop/has_unique_object_representations.compile.pass.cpp (-3)
diff --git a/libcxx/test/std/utilities/meta/meta.unary/meta.unary.comp/is_bounded_array.pass.cpp b/libcxx/test/std/utilities/meta/meta.unary/meta.unary.comp/is_bounded_array.pass.cpp
index 97e3afed1c036..8e57e8913dcbe 100644
--- a/libcxx/test/std/utilities/meta/meta.unary/meta.unary.comp/is_bounded_array.pass.cpp
+++ b/libcxx/test/std/utilities/meta/meta.unary/meta.unary.comp/is_bounded_array.pass.cpp
@@ -7,9 +7,6 @@
 //===----------------------------------------------------------------------===//
 // UNSUPPORTED: c++03, c++11, c++14, c++17
 
-// The Clang version that Android currently uses in the CI is too old.
-// UNSUPPORTED: LIBCXX-ANDROID-FIXME
-
 // type_traits
 
 // is_bounded_array<T>
diff --git a/libcxx/test/std/utilities/meta/meta.unary/meta.unary.prop/has_unique_object_representations.compile.pass.cpp b/libcxx/test/std/utilities/meta/meta.unary/meta.unary.prop/has_unique_object_representations.compile.pass.cpp
index ac63fec691377..bd7da40daf2bc 100644
--- a/libcxx/test/std/utilities/meta/meta.unary/meta.unary.prop/has_unique_object_representations.compile.pass.cpp
+++ b/libcxx/test/std/utilities/meta/meta.unary/meta.unary.prop/has_unique_object_representations.compile.pass.cpp
@@ -8,9 +8,6 @@
 
 // UNSUPPORTED: c++03, c++11, c++14
 
-// The Clang version that Android currently uses in the CI is too old.
-// UNSUPPORTED: LIBCXX-ANDROID-FIXME
-
 // type_traits
 
 // has_unique_object_representations

@rprichard rprichard merged commit 8aa4fc0 into llvm:main Jul 18, 2025
79 checks passed
@rprichard rprichard deleted the reenable-android-tests branch July 18, 2025 01:08
@ldionne ldionne added this to the LLVM 21.x Release milestone Jul 21, 2025
@github-project-automation github-project-automation bot moved this to Needs Triage in LLVM Release Status Jul 21, 2025
@ldionne
Copy link
Member

ldionne commented Jul 21, 2025

/cherry-pick 8aa4fc0

@llvmbot
Copy link
Member

llvmbot commented Jul 21, 2025

Failed to cherry-pick: 8aa4fc0

https://github.com/llvm/llvm-project/actions/runs/16422246220

Please manually backport the fix and push it to your github fork. Once this is done, please create a pull request

@tru tru moved this from Needs Triage to Needs Backport PR in LLVM Release Status Jul 22, 2025
frederick-vs-ja pushed a commit to frederick-vs-ja/llvm-project that referenced this pull request Jul 22, 2025
Now that the Android clang has been upgraded to clang-r563880
(llvm.org/pr148998), these two tests pass again.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
libc++ libc++ C++ Standard Library. Not GNU libstdc++. Not libc++abi. release:cherry-pick-failed
Projects
Status: Needs Backport PR
Development

Successfully merging this pull request may close these issues.

4 participants